#59 Alles Material löschen Funktion hinzugefügt #67
Conversation
Button um alles Material der Abteilung zu löschen auf der Material Seite der eigenen Abteilung hinzugefügt. Buttons für den Excel Import und Export auf die Material Seite der eigenen Abteilung verschoben. Vorher unter Abteilung => Settings. Kleine visuelle Verbesserungen.
There was a problem hiding this comment.
Ich habe noch einen Bug: Ich kann 1x das Material importieren, aber wenn ich dann nochmals auf den Upload button klicke, kann ich das File auswählen aber danach passiert nichts. Das Modal mit der Tabellauswahl öffnet sich nicht. Auf der alten dev Instanz funktioniert das. Könntest du das bitte noch beheben?
Vielen Dank für dein refactoring und die neue Funktion! Finde ich super!
| const { abteilung} = props; | ||
| const [updateLoading] = useState(false); | ||
|
|
||
| const delteMaterial = async () => { |
| @@ -124,6 +127,18 @@ export const AbteilungMaterialView = (props: AbteilungMaterialViewProps) => { | |||
| <AddStandortButton abteilungId={abteilung.id} /> | |||
| </Can> | |||
| </Col> | |||
| <Can I={'delete'} this={{ __caslSubjectType__: 'Material', abteilungId: abteilung.id } as AbteilungEntityCasl}> | |||
There was a problem hiding this comment.
Ich bin nicht wirklich ein Fan von den Buttons unter der Navigation. Gibt es da einen besseren Weg?
Habt ihr dazu eine Meinung @jan-preisig @JeremiasBachmann @pemko123 ?
There was a problem hiding this comment.
UM alles Material zu löschen denke ich sollte dies beim Excel Import sein oder nicht?
Gehört irgendwie zusammen. Wann sonst sollte man alles MAterial löschen?
There was a problem hiding this comment.
Wie wäre es mit einem Dropdown in der Navigation namens Aktionen von wo aus man verschiedene Aktionen ausführen kann? Entweder direkt aus dem Dropdwon das popup öffnen oder eine Subpage mit funktionen?
Oder Option 2 wäre man nimmt alles zu den Einstellungen und hat dort einerseits funkionen um diverse dinge zu Bearbeiten und Erstellen sowie die Einstellungen.
Findet ihr davon etwas gut? @JeremiasBachmann @Leo1212 @pemko123
There was a problem hiding this comment.
Aktionen in der Navigation könnte man machen, solange es nicht zu viele werden. Unter Settings fände ich die Buttons irgendwie nicht sehr intuitiv, da Ich Buttons welche mit dem Material zu tun haben unter dem Menupunkt Material erwarten würde und nicht bei Settings. Ich finde die Buttons aber eigentlich nicht schlecht platziert, vor allem da sie ohnehin nur für das MatTeam/Admins sichtbar sind. Eine weiter Möglichkeit wäre das man für das MatTeam einen eigenen Menupunkt erstellt, in welchem sie das Mat verwalten können, und MatTeam spezifische Einstellung wie z.B. das aktivieren von Email Benachrichtigungen etc. haben.
There was a problem hiding this comment.
UM alles Material zu löschen denke ich sollte dies beim Excel Import sein oder nicht? Gehört irgendwie zusammen. Wann sonst sollte man alles MAterial löschen?
Beim Excel Import wurde dies so umgesetzt, dass man wählen kann ob das neue Material, das alte Material ersetzten soll oder ob es zum alten Material hinzugefügt werden soll.
| > | ||
| Abbrechen | ||
| </Button>, | ||
| <Tooltip title="Bestehendes Material wird mit dem Inhalt der Excel-Tabelle ergänzt."> |
There was a problem hiding this comment.
Ich habe da noch ein Tooltip hinzugefügt, damit man genau weiss was passiert.
There was a problem hiding this comment.
Perfekt, Dankeschön.
| okText="Ja" | ||
| cancelText="Nein" | ||
| > | ||
| <Tooltip title="Bestehendes Material wird gelöscht und mit dem Inhalt der Excel-Tabelle ersetzt."> |
|
@JeremiasBachmann Ich kann gerne das Approven. Das mit den Buttons ist okay für mich. Aber bitte noch den Bug beheben ;) |
|
Hab den Bug jetzt noch gefixed |
|
Funktioniert. Vielen Dank @jan-preisig |
Leo1212
deleted the
feature/#59_button_um_alle_materialien_zu_loeschen
branch
Button um alles Material der Abteilung zu löschen auf der Material Seite der eigenen Abteilung hinzugefügt.
Änderung beim Importieren des Excels, muss nun ausgewählt werden ob das neue Material zum bestehenden Material der Abteilung hinzugefügt werden soll, oder ob das Bisherige Material ersetzt werden soll.
Buttons für den Excel Import und Export auf die Material Seite der eigenen Abteilung verschoben. Vorher unter Abteilung => Settings.
Kleine visuelle Verbesserungen.